Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prevent header plugin from turning code blocks into expressions #3510

Closed
wants to merge 2 commits into from

Conversation

matthewp
Copy link
Contributor

@matthewp matthewp commented Jun 2, 2022

Changes

Testing

  • Test added for this scenario

Docs

N/A, bug fix.

@changeset-bot
Copy link

changeset-bot bot commented Jun 2, 2022

🦋 Changeset detected

Latest commit: 378c16d

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@astrojs/markdown-remark Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@github-actions github-actions bot added the feat: markdown Related to Markdown (scope) label Jun 2, 2022
@natemoo-re
Copy link
Member

Thoughts on this approach vs the one in #3502?

@matthewp
Copy link
Contributor Author

matthewp commented Jun 2, 2022

It's a bit similar but there's seems to be handling some html case I wasn't handling. Also I'm not sure if theirs handles a case where there's only a code block and nothing else. I'll comment on their PR.

@matthewp matthewp closed this Jun 2, 2022
@bluwy bluwy deleted the code-block-rehype branch October 8, 2024 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feat: markdown Related to Markdown (scope)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 BUG: JSX-like expressions in code blocks are rendered incorrectly in headings
2 participants